Skip to content

feat: backport list endpoint for out-of-band records to 1.2.lts#4058

Merged
swcurran merged 1 commit intoopenwallet-foundation:1.2.ltsfrom
OpSecId:backport-oob-records-1.2.lts
Feb 23, 2026
Merged

feat: backport list endpoint for out-of-band records to 1.2.lts#4058
swcurran merged 1 commit intoopenwallet-foundation:1.2.ltsfrom
OpSecId:backport-oob-records-1.2.lts

Conversation

@PatStLouis
Copy link
Contributor

@PatStLouis PatStLouis commented Feb 19, 2026

Summary

Backport of #4046 to the 1.2.lts branch.

Adds GET /out-of-band/records endpoint to query OobRecord entries with pagination and filtering support. This addresses the lack of any way to discover or list existing OOB records in the wallet.

  • Supports filtering by state, role, connection_id, and invi_msg_id
  • Standard pagination parameters (limit, offset)
  • Includes 4 new unit tests

Note: The 1.2.lts branch uses get_limit_offset (limit/offset only) rather than get_paginated_query_params (which adds order_by/descending on main and 1.3.lts). Additionally, state and role are placed in post_filter rather than tag_filter since OobRecord.TAG_NAMES does not include state on this branch.

Closes #4044

Test plan

  • All 13 OOB route tests pass locally (4 new + 9 existing)
  • CI passes

Made with Cursor

Backport of b6c82ac from main. Adds GET /out-of-band/records endpoint
to query OobRecord entries with pagination and filtering support.

Supports filtering by state, role, connection_id, and invi_msg_id, plus
standard pagination parameters (limit, offset).

Closes openwallet-foundation#4044

Signed-off-by: Patrick St-Louis <patrick.st-louis@opsecid.ca>
Co-authored-by: Cursor <cursoragent@cursor.com>
@PatStLouis PatStLouis force-pushed the backport-oob-records-1.2.lts branch from c8d6660 to 5fe7a96 Compare February 19, 2026 03:23
@sonarqubecloud
Copy link

@PatStLouis PatStLouis requested a review from swcurran February 19, 2026 03:31
@swcurran swcurran merged commit 625d047 into openwallet-foundation:1.2.lts Feb 23, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants